Skip to content

[19.0][MIG] product_pricelist_revision: Migration to 19.0#2100

Open
BhaveshHeliconia wants to merge 28 commits intoOCA:19.0from
HeliconiaIO:19.0-mig-product_pricelist_revision
Open

[19.0][MIG] product_pricelist_revision: Migration to 19.0#2100
BhaveshHeliconia wants to merge 28 commits intoOCA:19.0from
HeliconiaIO:19.0-mig-product_pricelist_revision

Conversation

@BhaveshHeliconia
Copy link
Copy Markdown
Contributor

No description provided.

@BhaveshHeliconia BhaveshHeliconia mentioned this pull request Oct 15, 2025
73 tasks
@rousseldenis
Copy link
Copy Markdown
Contributor

/ocabot migration product_pricelist_revision

Copy link
Copy Markdown
Contributor

@gdgellatly gdgellatly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functional test on own db plus code review LGTM.

Copy link
Copy Markdown
Contributor

@gdgellatly gdgellatly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry I put review against wrong module. I now tested this one on live database and it didn't work. It duplicated the item just fine but did not take the variance. Will get back to tomorrow.

sergio-teruel and others added 23 commits March 6, 2026 14:12
[UPD] Update product_pricelist_revision.pot

[UPD] README.rst

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: product-attribute-12.0/product-attribute-12.0-product_pricelist_revision
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-12-0/product-attribute-12-0-product_pricelist_revision/
…t filter

TT28144

[UPD] Update product_pricelist_revision.pot
Currently translated at 100.0% (30 of 30 strings)

Translation: product-attribute-13.0/product-attribute-13.0-product_pricelist_revision
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-13-0/product-attribute-13-0-product_pricelist_revision/ca/
Before this change on runbot we caught a Warning about deprecated call to decimal_precision.get_precision(<application>)

[UPD] README.rst

product_pricelist_revision 13.0.1.1.0
Fix website key in the manifests
Currently translated at 31.0% (9 of 29 strings)

Translation: product-attribute-14.0/product-attribute-14.0-product_pricelist_revision
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-14-0/product-attribute-14-0-product_pricelist_revision/it/
Currently translated at 100.0% (29 of 29 strings)

Translation: product-attribute-16.0/product-attribute-16.0-product_pricelist_revision
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-16-0/product-attribute-16-0-product_pricelist_revision/it/
- Include context keys for avoiding mail operations overhead.
Currently translated at 100.0% (29 of 29 strings)

Translation: product-attribute-16.0/product-attribute-16.0-product_pricelist_revision
Translate-URL: https://translation.odoo-community.org/projects/product-attribute-16-0/product-attribute-16-0-product_pricelist_revision/fr/
@BhaveshHeliconia BhaveshHeliconia force-pushed the 19.0-mig-product_pricelist_revision branch from 1b96722 to 68cfb1b Compare March 6, 2026 08:42
@BhaveshHeliconia
Copy link
Copy Markdown
Contributor Author

Sorry I put review against wrong module. I now tested this one on live database and it didn't work. It duplicated the item just fine but did not take the variance. Will get back to tomorrow.

Hi @gdgellatly,

I have tested this module locally and on Runbot, and it is working as expected.

Please note that this module's duplicate wizard only works with fixed price items (compute_price = 'fixed'). It does not apply the variance for other pricing types such as formula or percentage/discount. The wizard view also mentions this:

"Note: Only works with fixed price items. The variation % will be applied to calculate the new price."

If you are facing any issue specifically with fixed price items, could you please elaborate on the steps to reproduce and where exactly you are getting stuck? That would help us investigate further.

@gdgellatly
Copy link
Copy Markdown
Contributor

gdgellatly commented Mar 8, 2026

Sorry I put review against wrong module. I now tested this one on live database and it didn't work. It duplicated the item just fine but did not take the variance. Will get back to tomorrow.

Hi @gdgellatly,

I have tested this module locally and on Runbot, and it is working as expected.

Please note that this module's duplicate wizard only works with fixed price items (compute_price = 'fixed'). It does not apply the variance for other pricing types such as formula or percentage/discount. The wizard view also mentions this:

"Note: Only works with fixed price items. The variation % will be applied to calculate the new price."

If you are facing any issue specifically with fixed price items, could you please elaborate on the steps to reproduce and where exactly you are getting stuck? That would help us investigate further.

Hi @BhaveshHeliconia, yes sorry we worked that out and deleted the module. We have no use case for it so will remove my review. EDIT: I no longr have access to remove reviews so we will just have to live with it, it did work, just not how expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.